feat: add album-record-store content template#115
Open
sweetmantech wants to merge 1 commit intomainfrom
Open
Conversation
New template for "album art on vinyl in a record store" content. Includes style guide, caption guide, example captions, video moods, and video movements. No code changes — template data only. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughAdded five new JSON template configuration files for an album-record-store feature, defining caption generation rules, visual styling directives, video mood descriptions, motion prompts, and example caption content to guide downstream rendering behavior. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/content/templates/album-record-store/style-guide.json`:
- Line 4: The template currently sets usesFaceGuide to true which conflicts with
the template's "no person in the shot" requirement; change the usesFaceGuide
boolean to false so face-guidance is disabled for this album template (update
the usesFaceGuide property in the template JSON to false and verify any
associated face-guidance flags or validators reference usesFaceGuide rather than
a separate flag).
- Line 5: The JSON key "customInstruction" contains contradictory directives
about text: change the wording to explicitly preserve any text that is part of
the original album art while removing only extraneous
overlays/captions/watermarks added to the image or scene; update the
"customInstruction" value to state “Do NOT alter or remove any text/design that
is part of the album cover art itself; however, remove any added captions,
watermarks, or overlays that are not part of the original artwork,” and keep the
rest of the scene instructions (vinyl on turntable, sleeve placement) unchanged
so callers know exactly what to preserve versus what to strip.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 79581e6e-9df5-4e09-b0a4-7d5977d10b02
📒 Files selected for processing (5)
src/content/templates/album-record-store/caption-guide.jsonsrc/content/templates/album-record-store/references/captions/examples.jsonsrc/content/templates/album-record-store/style-guide.jsonsrc/content/templates/album-record-store/video-moods.jsonsrc/content/templates/album-record-store/video-movements.json
| { | ||
| "name": "album-record-store", | ||
| "description": "Album cover art displayed in a gritty New York record store — vinyl spinning on a turntable", | ||
| "usesFaceGuide": true, |
There was a problem hiding this comment.
Set usesFaceGuide to false for this template.
Line 4 conflicts with Line 24 (“no person in the shot”). Keeping face guidance enabled can inject unintended identity constraints into a non-face composition.
Suggested change
- "usesFaceGuide": true,
+ "usesFaceGuide": false,📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "usesFaceGuide": true, | |
| "usesFaceGuide": false, |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/content/templates/album-record-store/style-guide.json` at line 4, The
template currently sets usesFaceGuide to true which conflicts with the
template's "no person in the shot" requirement; change the usesFaceGuide boolean
to false so face-guidance is disabled for this album template (update the
usesFaceGuide property in the template JSON to false and verify any associated
face-guidance flags or validators reference usesFaceGuide rather than a separate
flag).
| "name": "album-record-store", | ||
| "description": "Album cover art displayed in a gritty New York record store — vinyl spinning on a turntable", | ||
| "usesFaceGuide": true, | ||
| "customInstruction": "Place the album cover art from the first image onto a vinyl record sleeve and display it prominently in the scene. The album art should be clearly visible — propped up on a shelf, leaning against a crate, or displayed on the counter next to the turntable. A vinyl record from the same album is spinning on a turntable nearby. Do NOT alter the album art — reproduce it exactly. Remove any text, captions, watermarks, or overlays from the generated scene. The album art itself should retain its original text/design.", |
There was a problem hiding this comment.
Resolve conflicting text-preservation instructions in customInstruction.
Line 5 says both “Do NOT alter the album art” and “Remove any text…”. This is ambiguous and can lead to stripping text that is part of the cover art.
Suggested clarification
- "customInstruction": "Place the album cover art from the first image onto a vinyl record sleeve and display it prominently in the scene. The album art should be clearly visible — propped up on a shelf, leaning against a crate, or displayed on the counter next to the turntable. A vinyl record from the same album is spinning on a turntable nearby. Do NOT alter the album art — reproduce it exactly. Remove any text, captions, watermarks, or overlays from the generated scene. The album art itself should retain its original text/design.",
+ "customInstruction": "Place the album cover art from the first image onto a vinyl record sleeve and display it prominently in the scene. The album art should be clearly visible — propped up on a shelf, leaning against a crate, or displayed on the counter next to the turntable. A vinyl record from the same album is spinning on a turntable nearby. Do NOT alter the album art — reproduce it exactly, including its original text/design. Do not add any extra scene overlays, subtitles, captions, or watermarks outside the album artwork.",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "customInstruction": "Place the album cover art from the first image onto a vinyl record sleeve and display it prominently in the scene. The album art should be clearly visible — propped up on a shelf, leaning against a crate, or displayed on the counter next to the turntable. A vinyl record from the same album is spinning on a turntable nearby. Do NOT alter the album art — reproduce it exactly. Remove any text, captions, watermarks, or overlays from the generated scene. The album art itself should retain its original text/design.", | |
| "customInstruction": "Place the album cover art from the first image onto a vinyl record sleeve and display it prominently in the scene. The album art should be clearly visible — propped up on a shelf, leaning against a crate, or displayed on the counter next to the turntable. A vinyl record from the same album is spinning on a turntable nearby. Do NOT alter the album art — reproduce it exactly, including its original text/design. Do not add any extra scene overlays, subtitles, captions, or watermarks outside the album artwork.", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/content/templates/album-record-store/style-guide.json` at line 5, The
JSON key "customInstruction" contains contradictory directives about text:
change the wording to explicitly preserve any text that is part of the original
album art while removing only extraneous overlays/captions/watermarks added to
the image or scene; update the "customInstruction" value to state “Do NOT alter
or remove any text/design that is part of the album cover art itself; however,
remove any added captions, watermarks, or overlays that are not part of the
original artwork,” and keep the rest of the scene instructions (vinyl on
turntable, sleeve placement) unchanged so callers know exactly what to preserve
versus what to strip.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
loadTemplate("album-record-store")🤖 Generated with Claude Code
Summary by CodeRabbit